shortcutcontroller: Introduce

This is a very barebones controller that currently does nothing but
activate the binding signals. Yay.

And because we have bindings on every widget (Yes, a GtkGrid has a
keybinding - 2 in fact), we need that controller everywhere.

bindings: Make gtk_binding_parse_signal() use GVariantBuilder

And because it's the last user of GtkBindingArg and
gtk_binding_entry_add_signall(), also remove those two.

bindings: Parse into GVariantBuilder directly

Avoids the indirection via GtkBindingArg

bindings: Add gtk_binding_entry_add_signal_variant()

This function is the replacement for
gtk_binding_entry_add_signall().

The GVariant will be demarshalled and passed to the action signal upon
binding activation. The same rules apply as used to apply for
GtkBindingArg, in that long, double and string args are now replaced by
"x", "d" and "s" variant types.

bindings: Replace GtkBindingArg arguments with GVariant

So far, this only replaces invocation, not yet parsing.

Merge branch 'matthiasc/for-master2' into 'master'

Matthiasc/for master2

See merge request GNOME/gtk!1568

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1567

passwordentry: Remove unused include

entry: Remove unused include

text view: Fix touch selection

We forgot to allocated that popover.

event controller focus: drop include

This header is not needed here, and it is going away.

flattenlistmodel: Give access to child models

Add an api to retrieve the model containing a given
item in a flatten listmodel. This is useful when the
individual items in the list don't have backpointers.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1565

Merge branch 'wip/chergert/muxer-robustness-fix' into 'master'

muxer: force muxer dispoal as part of widget dispose

See merge request GNOME/gtk!1564

filechoosererrorstack: Fix object life-cycle

Unparent children in dispose, not in finalize.

testsuite: Set NO_AT_BRIDGE for gtk tests

The a11y stack insists on spewing hard to control warnings
that make our tests fail. So, no accessibility for gtk tests :(

muxer: force muxer dispoal as part of widget dispose

We need to cleanup state here immediately so that we do not potentially
access the g_class private data after it been finalized. This ensures that
the borrowed reference is dropped by the muxer.

Merge branch 'wip/sadiq/fix-leak' into 'master'

singleselection: Fix a memory leak

See merge request GNOME/gtk!1554

magnifier: Don't try to use a null widget paintable

We destroy the paintable in dispose, which can be called multiple times.

inspector: Fix child widget life cycle

unparent them in dispose

Merge branch 'patch-2' into 'master'

gtkfilechoosernative: Remove duplicate "the" in the documentation

See merge request GNOME/gtk!1557

gtkfilechoosernative: Remove duplicate "the" in the documentation

testsuite/gtk/flattenlistmodel.c: fix build

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>

singleselection: Fix a memory leak

3.98.1

a11y: Don't re turn uninitialized memory

For some reason, this was now showing up in the
a11y tests as random image sizes in the about
dialog test.

Merge branch 'filechooser-popover-fixes' into 'master'

Filechooser popover fixes

See merge request GNOME/gtk!1553

file chooser: Fix the rename file popover

Make this popover work again by attaching it
to the file chooser itself.

filechooser: Fix the file list popover

Make this popover work again.

filechooser: Fix the new folder popover

We were listening for change notification on a
no-longer-existing property.

places sidebar: Fix the popovers

We need to check-resize the popovers.

Merge branch 'portal-dir-chooser' into 'master'

filechoosernativeportal: Allowing selecting folder(s)

See merge request GNOME/gtk!1552

filechoosernativeportal: Allowing selecting folder(s)

Use the "directory" portal option when asked to select folders.

Merge branch 'portal-chooser-fail' into 'master'

portal file chooser: Handle errors

Closes #2527

See merge request GNOME/gtk!1551

portal file chooser: Handle errors

Fall back to showing a local file chooser
if we get DBus errors from the portal.

Closes: #2527

Merge branch 'fix-inspector-picking' into 'master'

inspector: Make picking work again

Closes #2528

See merge request GNOME/gtk!1549

Merge branch 'fix-inspector-key' into 'master'

x11: Fix key event state translation

See merge request GNOME/gtk!1550

x11: Fix key event state translation

In the event refactoring, a bit was lost.

This makes Ctrl-Shift-i work again to open
the inspector.

inspector: Make picking work again

We used to either make the inspector see- and click-thru
or lower it, but sadly translucency broke after we
removed surface opacity, and lowering doesn't work
on Wayland. So just hide the inspector window while
we are picking.

Fixes: #2528

Merge branch 'new-text-attributes' into 'master'

New text attributes

See merge request GNOME/gtk!1547

Make overline support conditional

The ci image does not have Pango 1.45, so for the
time being, make the overline support conditional
on Pango being new enough.

Require pango 1.44.4

This lets use use the insert-hyphen attribute
unconditionally.

gtk-demo: Add hyphenation control to the markup demo

gtk-demo: Allow word breaking in the markup demo

We want to show hyphenation control here.

textbuffer: Parse overline and hyphenation attributes

When inserting Pango markup into a text buffer, translate
Pango attributes for overlines and hyphenation control
into the corresponding text tag properties.

textlayout: Handle new text tag properties

Translate tag properties for overlines and
hyphenation control into Pango attributes.

texttag: Support overline and hyphenation control

Add support for recently added Pango attributes
for overlines and hyphenation control. The new
properties of GtkTextTag are
overline, overline-rgba, allow-breaks, show-spaces
and insert-hyphens.

Merge branch 'master_MR1290' into 'master'

Copy of !1290 for master

Closes #2365

See merge request GNOME/gtk!1546

Merge branch 'wip/chergert/fix-class-private-data-usage' into 'master'

Fix GtkWidgetClassPrivate data usage

See merge request GNOME/gtk!1545

widget: fix class private data usage to be _init() safe

Before this commit, adding GtkWidgetAction to class private data would
require copying the actions to each subclass as they were built or
modified. This was convenient in that it is a sort of "copy on write"
semantic.

However, due to the way that GTypeInstance works with base _init()
functions, the "g_class" pointer in GTypeInstance is updated as each
_init() function is called. That means you cannot access the subclasses
class private data, but only the parent class private data.

If instead we use a singly linked list of GtkWidgetAction, each subclass
has their own "head" yet all subclasses share the tail of the
GtkWidgetAction chain.

This creates one bit of complexity though. You need a stable way to know
which "bit" is the "enabled" bit of the action so we can track enabled
GAction state. That is easily solved by calculating the distance to the
end of the chain for a given action so that base classes sort ahead of
subclasses. Since the parent class always knows its parent's actions, the
position is stable.

A new dynamic bitarray helper also helps us avoid allocations in all the
current cases (up to 64 actions per widget) and dynamically switches to
malloc if that is to ever be exceeded.

widget: move class private data to gtkwidgetprivate.h

We want access to the private data from the action muxer so we can just
move the structures to the gtkwidgetprivate.h header. Alternatively we
could create accessors, but given that we'll probably need to use this
in other areas, seems reasonable to just put it there.

Merge branch 'test-protocol' into 'master'

Test protocol: TAP

See merge request GNOME/gtk!1543

testsuite: disable performance tests

For some reason, these tests are flaky in ci,
they always work locally for me. So, until
we use the data these tests produce for something,
lets just turn them off.

Merge branch 'wip/carlosg/text-handle-natives' into 'master'

Make text handles use native surfaces

See merge request GNOME/gtk!1487

gkimmulticontext: Fix crash due to leaked signal handler

The problem is caused by gtk_im_multicontext_set_slave(), which forgets
to disconnect these signal handlers:
* gtk_im_multicontext_retrieve_surrounding_cb
* gtk_im_multicontext_delete_surrounding_cb

If slave GtkImContext emits signal after GtkIMMulticontext context is
destroyed, this leads to reading freed memory, sometimes causing a crash.

Fixes: #2365

Merge branch 'ci-dbus-launch' into 'master'

ci: Add dbus-launch to the Docker build

See merge request GNOME/gtk!1544

testsuite: Fix an a11y test

The derive test wasn't producing TAP, so
running it as part of a tap testsuite wasn't
going well. Fix that.

testsuite: Tell meson we use TAP

This makes meson actually parse the individual test
results. Most of the time, it does not make a difference,
but one case where it does is when all the individual
tests of a binary are skipped, meson will mark the
test as skipped.

reftests: Fix an xfail to work

The background-image-multiple.ref.ui file uses
non-existing properties, which gives us a g_warning,
and the glib test framework insists on treating
warnings as fatal, so we end up doing exit(133),
which in turn makes the meson TAP parser ignore
its xfails.

Comment out the nonexisting properties, so we can
fail properly, and then in turn xfail properly.

testsuite: Don't exit unsuccessfully when using TAP

The meson TAP parser doesn't take this lightly and
forgets all about xfails when we exit(1), so don't.

ci: Add dbus-launch to the Docker build

gtktextview: Set magnifier popover on top

At the bottom, it sometimes has to fight for the same position than
text handles, besides might not be ergonomically convenient (eg.
finger/hand partly covering the popover). Move it at the top to fix
both.

gtktext: Set magnifier popover on top

At the bottom, it sometimes has to fight for the same position than
text handles, besides might not be ergonomically convenient (eg.
finger/hand partly covering the popover). Move it at the top to fix
both.

gtkpopover: Re-present surface if position/pointing-to change while visible

This ensures the popover will follow the new position parameters. This is
necessary for popovers like the text magnifier that can be repositioned
while visible.

gtkpopover: Ensure the pointed to rectangle has a minimum w/h

On wayland, protocol errors will be raised if the rectangle width/height
are 0. Avoid that situation so it's valid to all popovers.

gtkwindow: Remove internal popover API

This was only used by text handles, and not anymore.

gtktexthandle: Refactor and use native surfaces

Instead of being a GObject managing two GtkWidgets, make GtkTextHandle
a GtkWidget subclass, representing a single handle.

From the perspective of users (GtkText and GtkTextView), this is not a
big leap since they have to be aware of a great deal of text handles'
state. It actually makes things more direct and simple.

With text handles being widgets, those can be actual children of the
widget, and may have their own GdkSurface that we move around at will.
This is the second major aspect of this refactor.

reftests: Don't run tests with --verbose

Unexpected output tends to break TAP.

gsk: Don't unconditionally spew to stdout

testsuite: Don't g_print from tests

This breaks the TAP output.

testsuite: Fix the accessible test

This test was happily doing nothing.

Merge branch 'disable-window-test-2' into 'master'

Disable window test 2

See merge request GNOME/gtk!1541

temporarily disable window test

This test was broken by the GdkToplevel refactoring,
and needs a big gtkwindow.c configure request cleanup
before it will work again.

Revert "Merge branch 'disable-window-test' into 'master'"

This reverts commit 3ac4c76b18cc89a841ce09f0943539f16988fd21, reversing
changes made to 6ec96d2e989d029a303b8b20ec72b86f974c0e87.

Merge branch 'disable-window-test' into 'master'

Disable window test

See merge request GNOME/gtk!1540

Fix the action test

We have a test that enumerates the GtkText actions,
so when a new open appears, the test needs to be updated.

Fix a compiler warning

temporarily disable window test

This test was broken by the GdkToplevel refactoring,
and needs a big gtkwindow.c configure request cleanup
before it will work again.

text view: Fix touch selection

We forgot to allocated that popover.

widget: Drop the ::popup-menu signal

This is now done in widgets which have context
menus.

placesview: Stop using ::popup-menu

This signal is going away.

filechooser: Stop using ::popup-menu

This signal is going away.

mountoperation: Stop using ::popup-menu

We can just use a shortcut controller directly.

gtk-demo: Stop emitting ::popup-menu

The signal was not used anyway, in the font explorer demo.

colorchooser: Stop using ::popup-menu

This signal is going away. Use an action instead.

range: Remove ::popup-menu emission

This signal is going away, and having context menus
on sliders is not really a thing anyway.

scrollbar: Remove :popup-menu forwarding

This signal is going away.

emojichooser: Stop using ::popup-menu

This signal is going away. Use an action instead.

Use an action for the context menu keybinding

The ::popup-menu signal is going away.

Print mnemonic triggers clearly

inspector: Show shortcuts

At a tab that lists the shortcuts contained in a
GtkShortcutController.

Only create a class shortcut controller if we have shortcuts

No point in creating objects that just hold empty lists.

widget: Name the controllers

window: Name the the controllers

shortcutmanager: Name the controllers

This is helpful in the inspector.

Move shortcut manager initialization code

It is just too ugly to use quarks across multiple
source files, so add a private helper function that
attaches the controllers.

filechooser: Trigger the location popup via bindings

Simplifies code quite a bit.

accels: Remove GtkAccelGroup

testmenubutton: Don't create a GtkAccelGroup

It's unused.

testsuite: Remove GtkAccelGroup usage